Skip to content

Add wasm32-unknown-emscripten platform support#1628

Draft
MaxDesiatov wants to merge 1 commit intoswiftlang:mainfrom
MaxDesiatov:maxd/emscripten
Draft

Add wasm32-unknown-emscripten platform support#1628
MaxDesiatov wants to merge 1 commit intoswiftlang:mainfrom
MaxDesiatov:maxd/emscripten

Conversation

@MaxDesiatov
Copy link
Copy Markdown
Contributor

[One line description of your change]

Motivation:

[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]

Modifications:

[Describe the modifications you've done.]

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@grynspan
Copy link
Copy Markdown
Contributor

I would expect this to be tracked by targetEnvironment() rather than os() but okay.

If it is the intent of the Platform Steering Group to support Emscripten, we need CI support for it as well and some way to test changes.

target_link_libraries(Testing PUBLIC
Foundation)
if(NOT CMAKE_SYSTEM_NAME STREQUAL Emscripten)
target_link_libraries(Testing PUBLIC
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is Foundation not available in this environment? Is this a temporary constraint?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is marked as draft, as it's not ready for review yet.

@grynspan grynspan added wasi/wasm 🧭 WebAssembly support new-platform-support 🌟 Boldly go where no `#expect` macro has gone before (to a new platform) labels Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-platform-support 🌟 Boldly go where no `#expect` macro has gone before (to a new platform) wasi/wasm 🧭 WebAssembly support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants